Skip to content

Fix problems from cdeps1.0.94#405

Merged
billsacks merged 4 commits into
ESCOMP:mainfrom
ekluzek:fix_probs_from_1094
May 20, 2026
Merged

Fix problems from cdeps1.0.94#405
billsacks merged 4 commits into
ESCOMP:mainfrom
ekluzek:fix_probs_from_1094

Conversation

@ekluzek
Copy link
Copy Markdown
Collaborator

@ekluzek ekluzek commented May 9, 2026

Description of changes

Fix the problems that @billsacks noticed after cdesp1.0.94 came in. These were both things that I was embarrsesed to have let gone through.

Specific notes

Contributors other than yourself, if any: @billsacks

CDEPS Issues Fixed (include github issue #):
Fixes #402

Are there dependencies on other component PRs (if so list): No

Are changes expected to change answers (bfb, different to roundoff, more substantial): b4b

Any User Interface Changes (namelist or namelist defaults changes): No

Testing performed (e.g. aux_cdeps, CESM prealpha, etc):
running aux_cdeps

Hashes used for testing:
submodules from: cesm3_0_beta08

@ekluzek ekluzek requested a review from billsacks May 9, 2026 00:51
@ekluzek ekluzek added enhancement New feature or request CESM Only labels May 9, 2026
@ekluzek ekluzek added answers are bfb Responsibility: CTSM Responsibility to manage and accomplish this issue is the CTSM Software group labels May 9, 2026
Copy link
Copy Markdown
Member

@billsacks billsacks left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for these fixes, @ekluzek . I see this is marked as draft. Let me know when you feel it's ready to come in.

@ekluzek ekluzek self-assigned this May 11, 2026
@ekluzek ekluzek moved this from Todo to In Progress in LMWG: Sprint Planning Board May 11, 2026
@wwieder wwieder mentioned this pull request May 20, 2026
4 tasks
@billsacks billsacks marked this pull request as ready for review May 20, 2026 19:50
@billsacks
Copy link
Copy Markdown
Member

@ekluzek - I'm going ahead and merging this even though you had marked it as a draft, because it seems good to get these fixes in. If there was more you still wanted to do, that can come in a follow-up PR.

@billsacks billsacks merged commit 5d9d15b into ESCOMP:main May 20, 2026
1 check passed
@github-project-automation github-project-automation Bot moved this from In Progress to Done in LMWG: Sprint Planning Board May 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

answers are bfb CESM Only enhancement New feature or request Responsibility: CTSM Responsibility to manage and accomplish this issue is the CTSM Software group

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants